Skip to content

Conversation

@Be-ing
Copy link
Contributor

@Be-ing Be-ing commented Oct 19, 2025

Remove the use_vaapi and use_nvidia Cargo features. Instead:

Always require libva on x86 Linux, checked with pkg-config.

If cuda.h exists, build with NVidia hardware acceleration support. If you don't have an NVidia GPU, you probably don't want to install a large proprietary driver. The only reason to have the CUDA Toolkit installed if you don't have an NVidia GPU is if you're building a binary that might be used on machines with an NVidia GPU.

Also, allow specifying a non-default path for the CUDA Toolkit with the CUDA_HOME environment variable.

@Be-ing Be-ing force-pushed the hw_acceleration_build branch 3 times, most recently from d03901a to b69e0d1 Compare October 19, 2025 21:36
Remove the use_vaapi and use_nvidia Cargo features. Instead:

Always require libva on x86 Linux, checked with pkg-config.

If cuda.h exists, build with NVidia hardware acceleration support.
If you don't have an NVidia GPU, you probably don't want to
install a large proprietary driver. The only reason to have the
CUDA Toolkit installed if you don't have an NVidia GPU is if you're
building a binary that might be used on machines with an NVidia
GPU.

Also, allow specifying a non-default path for the CUDA Toolkit
with the CUDA_HOME environment variable.
@Be-ing Be-ing force-pushed the hw_acceleration_build branch from b69e0d1 to 7b1e4ce Compare October 19, 2025 22:02
Copy link
Contributor

@cloudwebrtc cloudwebrtc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@cloudwebrtc cloudwebrtc merged commit 7ef7f6e into livekit:main Oct 27, 2025
5 checks passed
@github-actions github-actions bot mentioned this pull request Oct 27, 2025
@Be-ing Be-ing deleted the hw_acceleration_build branch October 27, 2025 03:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants